From 6ffbb5520a3a434a050ba4b5606e80423c04c924 Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Sun, 12 Jun 2011 21:27:45 -0700 Subject: [PATCH] * editfns.c (Ftranslate_region_internal): Omit redundant test. --- src/ChangeLog | 2 ++ src/editfns.c | 7 ++----- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/src/ChangeLog b/src/ChangeLog index a26c3b262f6..4d90956735b 100644 --- a/src/ChangeLog +++ b/src/ChangeLog @@ -1,5 +1,7 @@ 2011-06-13 Paul Eggert + * editfns.c (Ftranslate_region_internal): Omit redundant test. + * fns.c (concat): Minor tuning based on overflow analysis. This doesn't fix any bugs. Use int to hold character, instead of constantly refetching from Emacs object. Use XFASTINT, not diff --git a/src/editfns.c b/src/editfns.c index bcf3b751aa9..12e82c428ad 100644 --- a/src/editfns.c +++ b/src/editfns.c @@ -3088,14 +3088,11 @@ It returns the number of characters changed. */) } else { - int c; - nc = oc; val = CHAR_TABLE_REF (table, oc); - if (CHARACTERP (val) - && (c = XFASTINT (val), CHAR_VALID_P (c, 0))) + if (CHARACTERP (val)) { - nc = c; + nc = XFASTINT (val); str_len = CHAR_STRING (nc, buf); str = buf; } -- 2.30.2